Skip to content

[PWGJE] Add jetHadronsPID#16012

Open
llorenc-11 wants to merge 28 commits into
AliceO2Group:masterfrom
llorenc-11:PID-Hadrons-in-Jets
Open

[PWGJE] Add jetHadronsPID#16012
llorenc-11 wants to merge 28 commits into
AliceO2Group:masterfrom
llorenc-11:PID-Hadrons-in-Jets

Conversation

@llorenc-11
Copy link
Copy Markdown

@llorenc-11 llorenc-11 commented Apr 28, 2026

The reason for this pull request is to create a new analysis of PID in jets as a student's team with the collaboration of Warsaw University of Technology Physics department .
This is the first pull request from this account, I hope all the technical requirements are met.
If you have any questions, please contact me on leonard.lorenc@cern.ch

@github-actions github-actions Bot added the pwgje label Apr 28, 2026
@github-actions github-actions Bot changed the title [PWGJE] Adding file jetHadronsPID.cxx and it's workflow [PWGJE] Adding file jetHadronsPID.cxx and it's workflow Apr 28, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 28, 2026 •
edited
Loading

O2 linter results: ❌ 1 errors, ⚠️ 0 warnings, 🔕 0 disabled

@alibuild alibuild mentioned this pull request Apr 28, 2026
Closed
@llorenc-11 llorenc-11 marked this pull request as draft April 28, 2026 18:42
@llorenc-11 llorenc-11 marked this pull request as ready for review April 28, 2026 19:10
@vkucera
Copy link
Copy Markdown
Collaborator

vkucera commented Apr 29, 2026

@MyFavoriteGitHub your username does not really identify you well in the collaboration.

vkucera
vkucera reviewed Apr 29, 2026
View reviewed changes
Comment thread PWGJE/Tasks/jetHadronsPid.cxx Outdated
vkucera
vkucera reviewed Apr 29, 2026
View reviewed changes
Comment thread PWGJE/Tasks/jetHadronsPid.cxx Outdated
vkucera
vkucera reviewed Apr 29, 2026
View reviewed changes
Comment thread PWGJE/Tasks/jetHadronsPid.cxx
vkucera
vkucera reviewed Apr 29, 2026
View reviewed changes
Comment thread PWGJE/Tasks/jetHadronsPid.cxx Outdated
vkucera
vkucera reviewed Apr 29, 2026
View reviewed changes
Comment thread PWGJE/Tasks/jetHadronsPid.cxx Outdated
vkucera
vkucera reviewed Apr 29, 2026
View reviewed changes
Comment thread PWGJE/Tasks/jetHadronsPid.cxx Outdated
vkucera
vkucera reviewed May 28, 2026
View reviewed changes
Comment thread PWGJE/Tasks/jetHadronsPid.cxx Outdated
@llorenc-11 llorenc-11 marked this pull request as draft May 31, 2026 19:53
@llorenc-11 llorenc-11 marked this pull request as ready for review June 4, 2026 08:58
@llorenc-11 llorenc-11 requested a review from vkucera June 4, 2026 08:58
@vkucera
Copy link
Copy Markdown
Collaborator

vkucera commented Jun 4, 2026

@llorenc-11 Thanks for the updates. Since you are adding a new task, please make sure the PR does not introduce any errors or warnings.

@llorenc-11 llorenc-11 requested review from a team, ddobrigk, dsekihat, iarsene, jgrosseo and ktf as code owners June 4, 2026 11:49
@alibuild alibuild mentioned this pull request Jun 4, 2026
Closed
@llorenc-11 llorenc-11 changed the title [PWGJE] Adding file jetHadronsPID.cxx and it's workflow [PWGJE] Adding file JetHadronsPID.cxx and it's workflow Jun 4, 2026
@llorenc-11 llorenc-11 changed the title [PWGJE] Adding file JetHadronsPID.cxx and it's workflow [Infrastructure,PWGJE] Adding file JetHadronsPID.cxx and it's workflow Jun 4, 2026
@vkucera
Copy link
Copy Markdown
Collaborator

vkucera commented Jun 4, 2026

Please revert your commit modifying the MegaLinter configuration. If you want to update your branch, do it properly, according to the documentation.

@vkucera vkucera marked this pull request as draft June 4, 2026 12:06
@llorenc-11 llorenc-11 changed the title [Infrastructure,PWGJE] Adding file JetHadronsPID.cxx and it's workflow [PWGJE] Adding file JetHadronsPID.cxx and it's workflow Jun 4, 2026
@llorenc-11 llorenc-11 marked this pull request as ready for review June 4, 2026 13:07
@llorenc-11
Copy link
Copy Markdown
Author

llorenc-11 commented Jun 4, 2026

Sorry, I forgot I should have merged with upstream. I think the o2linter has problems with some other workflows, but those tasks already exist, so I did not change them.

@vkucera
Copy link
Copy Markdown
Collaborator

vkucera commented Jun 4, 2026

Sorry, I forgot I should have merged with upstream. I think the o2linter has problems with some other workflows, but those tasks already exist, so I did not change them.

There are still warnings caused by your code.

@vkucera vkucera changed the title [PWGJE] Adding file JetHadronsPID.cxx and it's workflow [PWGJE] Add jetHadronsPID Jun 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alibuild alibuild Awaiting requested review from alibuild alibuild is a code owner

@maoyx maoyx Awaiting requested review from maoyx maoyx is a code owner

@raymondEhlers raymondEhlers Awaiting requested review from raymondEhlers raymondEhlers is a code owner

@fjonasALICE fjonasALICE Awaiting requested review from fjonasALICE fjonasALICE is a code owner

@mfasDa mfasDa Awaiting requested review from mfasDa mfasDa is a code owner

@mhemmer-cern mhemmer-cern Awaiting requested review from mhemmer-cern mhemmer-cern is a code owner

@nzardosh nzardosh Awaiting requested review from nzardosh nzardosh is a code owner

@vkucera vkucera Awaiting requested review from vkucera

@jgrosseo jgrosseo Awaiting requested review from jgrosseo

@iarsene iarsene Awaiting requested review from iarsene

@ktf ktf Awaiting requested review from ktf

@ddobrigk ddobrigk Awaiting requested review from ddobrigk

@dsekihat dsekihat Awaiting requested review from dsekihat

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

pwgje

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@llorenc-11 @vkucera @nzardosh @hzalewski